Skip to content

Feature/pc latency diagnostics #470

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 48 commits into from
Aug 6, 2025
Merged

Conversation

markgalvan-intel
Copy link
Collaborator

Enabling MsBetweenSimulationStart and PCLatency metrics.

wdebkows-intel and others added 30 commits January 30, 2025 09:28
…w breaks to prevent incorrect fallthrough. Updated console application output.
…w breaks to prevent incorrect fallthrough. Updated console application output.
Generalized ExtractAppTimingData function. Added tracking of OutOfBoundPresentStarts. Added tracking of last sim start time in swap chain.
Added in initial PCL support. Also added better tracking for app provided data for both the PresentMon app provider events and PCL.
If both pcl and pm app provided events are present use pcl for metric calculations for msbetweensimulation and animationerror.
In non-fg cases the out of band present start time may not be sent. Use the pcl present start time event as a backup.
Needed to know when transitioning from cpu start based animation error to pcl animation error.
Still WIP as we need to track if there is any FlipDelayData between consume frames calls
Copy link
Collaborator

@planetchili planetchili left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, need to follow up on the build warning and the git changes churn for generated sources etc.

@planetchili planetchili merged commit f9f9b85 into main Aug 6, 2025
@planetchili planetchili deleted the feature/pc-latency-diagnostics branch August 6, 2025 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants